You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This PR allows users to sample functions from some Gaussian random field with Matern covariance.
I'm no better at statistics than when I wrote this code (a long time ago) and someone should probably check the properties.
Still needs some tests and there has to be a better way of doing the boundary thing* @connorjward? @UZerbinati does netgen handle this kind of thing?
For meshes with a boundary, the mesh needs to be grown by the size of the correlation length and then needs to be trimmed back to the original mesh. I think this might be quite a hard thing to do in general.
I think extending the boundary regions is a meshing/geometry problem which Firedrake isn't very well-suited to fixing. Netgen or GMSH are probably the way to go.
I also wonder if @ksagiyam's submesh stuff would be useful for handling the inner mesh.
@JDBetteridge are you looking to work on this yourself? Or are you hoping someone else can take over? If the latter then it would be extremely useful if you could provide some tests (that don't necessarily need to pass) that demonstrate usage and expected behaviour.
I'm intending to hand this over, but my intention is to provide tests. I have some tests already, but they need refactoring into py-test tests. Currently they are trying to assess statistics of these fields and are not suitable for the test suite as they take far too long to run!
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
JDBetteridge
force-pushed
the
JDBetteridge/randomfields_rebased
branch
from
Description
Anyone remember this fossil?
This PR allows users to sample functions from some Gaussian random field with Matern covariance.
I'm no better at statistics than when I wrote this code (a long time ago) and someone should probably check the properties.
Still needs some tests and there has to be a better way of doing the boundary thing* @connorjward? @UZerbinati does netgen handle this kind of thing?
some_notes.pdf